Add dsb component#588
Conversation
| namespace: protein_processing | ||
| description: "Filter background and foreground signals for normalising protein expression with DSB (Denoised and Scaled by Background)." | ||
| authors: | ||
| - name: Xichen Wu |
There was a problem hiding this comment.
Use the author yaml to insert author information
| - name: Xichen Wu | |
| - __merge__: /src/authors/xichen_wu.yaml | |
| roles: [ maintainer, author ] |
| namespace: protein_processing | ||
| description: "Normalize protein expression with DSB (Denoised and Scaled by Background)." | ||
| authors: | ||
| - name: Xichen Wu |
There was a problem hiding this comment.
Use the author yaml to insert author information
| - name: Xichen Wu | |
| - __merge__: /src/authors/xichen_wu.yaml | |
| roles: [ maintainer, author ] |
| - name: "--data_raw" | ||
| type: file | ||
| required: true | ||
| description: "A ``MuData`` object containing raw (unfiltered, including empty droplets) data for both ``prot`` and ``rna`` modalities." |
There was a problem hiding this comment.
We typically use --input to denote an input mudata object in openpipelines
| - name: "--data_raw" | |
| type: file | |
| required: true | |
| description: "A ``MuData`` object containing raw (unfiltered, including empty droplets) data for both ``prot`` and ``rna`` modalities." | |
| - name: "--input" | |
| type: file | |
| required: true | |
| description: "A ``MuData`` object containing raw (unfiltered, including empty droplets) data for both ``prot`` and ``rna`` modalities." | |
| example: input.h5mu |
| type: file | ||
| direction: output | ||
| description: dsb_index output directory | ||
| example: "dsb_output" |
There was a problem hiding this comment.
Make this a required argument and add a slash to the example to denote it's a directory
| example: "dsb_output" | |
| example: "dsb_output/" | |
| required: true |
| multiple: true | ||
| - name: "--output" | ||
| type: file | ||
| direction: output |
There was a problem hiding this comment.
I'd be inclined to make the output an h5mu file instead of a directory, because then it plays much nicer with the other components in OpenPipeline. If you want, we can discuss this in the next OpenPipelines working meeting.
|
Hi @wxicu ! Just following up on this PR. Do you have any thoughts about the comments I placed? |
Hey really sorry for my late response. Finally have time now to contribute on this. I will go through all the comments and update it soon! Again, really sorry for the delay |
|
Hi @wxicu ! Would you happen to have time to finalise this PR before the end of the year? Thanks! |
2 participants
Changelog
Add dsb component: which splits the dsb function into two components, the first takes care of recognising the barcodes for foreground and the background, while the second does the actual normalization
Checklist before requesting a review
I have performed a self-review of my code
Conforms to the Contributor's guide
Check the correct box. Does this PR contain:
Proposed changes are described in the CHANGELOG.md
CI tests succeed!